Skip to content

Add curly rule to ESLint configuration#2189

Merged
jakebailey merged 1 commit into
microsoft:mainfrom
Bashamega:eslint
Oct 5, 2025
Merged

Add curly rule to ESLint configuration#2189
jakebailey merged 1 commit into
microsoft:mainfrom
Bashamega:eslint

Conversation

@Bashamega

Copy link
Copy Markdown
Contributor

No description provided.

@saschanaz

Copy link
Copy Markdown
Contributor

I'm happy with this.

@jakebailey

Copy link
Copy Markdown
Member

I thought these rules had been removed from eslint proper? I guess this repo is very behind on linting versions?

@saschanaz

Copy link
Copy Markdown
Contributor

Hmm? It's a thing? https://eslint.org/docs/latest/rules/curly

@HolgerJeromin

HolgerJeromin commented Oct 5, 2025

Copy link
Copy Markdown
Contributor

I thought these rules had been removed from eslint

curly is not part of the stylistics rules which were migrated to another team.

https://eslint.style/rules

@jakebailey

Copy link
Copy Markdown
Member

Interesting choice to only get rid of some style rules...

@jakebailey

Copy link
Copy Markdown
Member

Personally I think doing prettier or dprint would be better solutions, but, it's all the same

@jakebailey
jakebailey merged commit 9b934cb into microsoft:main Oct 5, 2025
7 checks passed
@HolgerJeromin

Copy link
Copy Markdown
Contributor

@saschanaz

Copy link
Copy Markdown
Contributor

We do use prettier already, except it didn't say anything here.

@Bashamega
Bashamega deleted the eslint branch October 6, 2025 02:53
@HolgerJeromin

Copy link
Copy Markdown
Contributor

@saschanaz

Copy link
Copy Markdown
Contributor

If we used prettier only, yeah. Not much point if we have eslint IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants